gdm/util: Do not run dispose on user verifier

This is not a good habit anywhere and especially in JS code, I assume
the rationale for this was to ensure that cached instances of the proxy
were cleared, but this did not work properly (as it happened on
finalize), and it would still require [1].

So let's just avoid this and assume that the getters just return what we
expect.

[1] https://gitlab.gnome.org/GNOME/gdm/-/merge_requests/321

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3885>

diff --git a/js/gdm/util.js b/js/gdm/util.js
index 3334d61f0e8b74e27eb8edb69ac45568c7fdca3b..d7339c14056b04017096137c27c7b18fb0f8593e 100644 (file)
--- a/js/gdm/util.js
+++ b/js/gdm/util.js
@@ -201,12 +201,8 @@ export class ShellUserVerifier extends Signals.EventEmitter {
         if (this._userVerifier) {
             this._disconnectSignals();
             this._userVerifier.get_connection().disconnectObject(this);
-            this._userVerifier.run_dispose();
             this._userVerifier = null;
-            if (this._userVerifierChoiceList) {
-                this._userVerifierChoiceList.run_dispose();
-                this._userVerifierChoiceList = null;
-            }
+            this._userVerifierChoiceList = null;
         }
     }